Skip to content

Config changes for custom scopes support #1166

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
Tejas-Kochar merged 8 commits into from
Jan 14, 2026
Merged

Config changes for custom scopes support #1166

Tejas-Kochar merged 8 commits into from
Jan 14, 2026
+134 −11

Conversation

@Tejas-Kochar
Copy link
Contributor

@Tejas-Kochar Tejas-Kochar commented Dec 23, 2025 •
edited
Loading

Summary

  • change type of scopes from str to list.
  • add option for disabling refresh token to be used in U2M.
  • sort scopes when config is initialized.
  • add getters for scopes that add default scope "all-apis" if scopes is empty.

How is this tested?

  • Unit tests for loading scopes from a test .databrickscfg config file
  • Unit tests ensuring disable_oauth_refresh_token is set when environment variable DATABRICKS_DISABLE_OAUTH_REFRESH_TOKEN is set.

NO_CHANGELOG=true

@Tejas-Kochar Tejas-Kochar self-assigned this Dec 23, 2025
@Tejas-Kochar Tejas-Kochar force-pushed the custom-scopes branch 2 times, most recently from b50398b to 4fb6bd0 Compare December 25, 2025 14:26
Tejas-Kochar
Tejas-Kochar commented Jan 8, 2026
View reviewed changes
@Tejas-Kochar Tejas-Kochar requested review from hectorcast-db and removed request for parthban-db January 12, 2026 11:54
hectorcast-db
hectorcast-db reviewed Jan 12, 2026
View reviewed changes
hectorcast-db
hectorcast-db reviewed Jan 12, 2026
View reviewed changes
Copy link
Contributor

@hectorcast-db hectorcast-db left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few more tests we could add:

  • List as input
  • Decuplication
  • Edge cases: Empty strings, whitespace-only strings, None values
  • Space separated input (backwards compatibility)

@github-actions
Copy link

github-actions bot commented Jan 13, 2026

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/sdk-py

Inputs:

  • PR number: 1166
  • Commit SHA: a723ca836b32d855b074aa2b8770681f812c25fb

Checks will be approved automatically on success.

@Tejas-Kochar Tejas-Kochar added this pull request to the merge queue Jan 14, 2026
Merged via the queue into main with commit 321ca70 Jan 14, 2026
17 checks passed
@Tejas-Kochar Tejas-Kochar deleted the custom-scopes branch January 14, 2026 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hectorcast-db hectorcast-db hectorcast-db approved these changes

Assignees

@Tejas-Kochar Tejas-Kochar

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Tejas-Kochar @hectorcast-db